-
Notifications
You must be signed in to change notification settings - Fork 189
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
adding detached tag mode to stream #1189
Conversation
This is OK but it should probably also have the corresponding detached methods on |
added- let's see if tests pass now... |
This looks good now, however it's a breaking change, so it will have to be part of the next release cycle (which we'll likely be starting soon) |
Cool- is there anything else I can do around RustCrypto that might be good
for a beginner to this codebase? Always looking for weekend/downtime fun
code besides, like, project euler.
…On Sun, Jan 15, 2023 at 1:41 PM, Tony Arcieri ***@***.***> wrote:
This looks good now, however it's a breaking change, so it will have to be
part of the next release cycle (which we'll likely be starting soon)
—
Reply to this email directly, view it on GitHub
<#1189 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ACDPMJE7EEEJFMZHBQRMLZ3WSRHFFANCNFSM6AAAAAAT2CI45E>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
I also wrote a silly little Writer impl for Encryptor: can obviously be improved in multiple places (generalizing over ciphers, for one, haha), but it works here's how i used it to make my code a little nicer...
and here's the code:
|
sorta like #641 but not for hashes |
@laudiacay yeah, the buffering is what makes that tricky, but also without either fixed size chunks or some kind of framing protocol you can't write a corresponding decryptor because you don't know the boundaries of the AEAD messages (I personally prefer the fixed-sized chunks approach because it has no size overhead beyond the tags) If you're looking for other small things to work on, we could use some help here: RustCrypto/hashes#87 |
What is the status of this PR? I have a use case where I need to use a Also re:
Not sure if that has already come and gone, or was delayed since that was 6 months ago. So this may be a consideration as well. |
Still waiting on the next breaking release cycle, which will come after #1174 |
|
Hiiii
I really wanted detached tags on the streaming encryption mode because I was having to do some really messed up things with buffers that were slowing me down.
I wrote it, it compiles and I'm not sure what tests you'd want... I didn't quite get it working with y'all's macros either but I don't think that made sense bc they were meant to deduplicate code between Encryptor and Decryptor, but it only really makes sense for Encryptors to detach tags... Hope this is cleaned up okay enough, lmk what I should change :)